Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Initial ZF2 RBAC Component #2616

Closed
wants to merge 1 commit into from
Closed

Initial ZF2 RBAC Component #2616

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Sep 29, 2012

No description provided.

@ghost
Copy link
Author

ghost commented Sep 29, 2012

@HarryR
Copy link

HarryR commented Oct 1, 2012

While this RBAC interface feels better designed and implemented than the ZF2 ACL module, I really do hope the SpiffySecurity firewall interface makes it into ZF2.

fnmatch() for route matches makes makes it very easy to use and a standard interface for 'Identities' (as implemented in SpiffySecurity) would provide a nice way to tie the less than stellar Zend\Authentication and ZfcUser components in with the rest of the framework.

$0.02.

@ghost
Copy link
Author

ghost commented Oct 1, 2012

I'm not sure where or how that would be implemented. Furthermore, shouldn't firewalls be restricted to userland code?

@weierophinney
Copy link
Member

@HarryR Zend\Authentication and Zend\Permissions\Acl were designed from the outset to be standalone components that could work with or without the MVC. ACL is a very different paradigm than RBAC, and as such, it's neither better nor worse (the design is actually quite elegant, and one of the better ACL implementations I've seen. No I did not write it.).

I would prefer that RBAC follows the footsteps of Authentication and ACL, and be de-coupled from the MVC. If you want to implement something to do automated checks based on the current identity and/or route, the proper place to do that is at the integration layer -- in other words, a module.

@ghost
Copy link
Author

ghost commented Oct 2, 2012

This is ready for review.

@weierophinney
Copy link
Member

Awesome -- thanks, @spiffyjr !

@ghost ghost assigned weierophinney Oct 4, 2012
weierophinney added a commit to zendframework/zend-permissions-rbac that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants